Conversation
|
Начал падать тест AddInlineClassHandlerWhenTypeNotImplementedInterfaceShouldBeException Exception нужно заменить на ArgumentException |
| /// Ограничитель спама логов. | ||
| /// </summary> | ||
| private DateTime LastErrorPollingDate; | ||
| private DateTime _lastErrorPollingDate; |
There was a problem hiding this comment.
По приватным полям давай без "_", в проекте пытаюсь использовать так
private DateTime lastErrorPollingDate;
Подумаю как проще сделать, чтобы к единому формату привести код стайл. Может какой-нибудь файлик будет с описанием, либо если получится автоматом сделаю.
|
Файл с указанием версии думаю можно не править, потом перед релизом сам поправлю. |
| /// Обработка update через polling. | ||
| /// </summary> | ||
| public async Task UpdatePolling() | ||
| public async Task UpdatePolling(CancellationToken cancellationToken = default) // TODO: добавить постфикс Async. P.S. метод больше похож на приватный. Если приватный - убрать default для cT. |
There was a problem hiding this comment.
Согласен, это должен быть приватный
|
TODO названий можно сразу поправить, единственное попрошу в PR добавить информацию про эти методы, чтобы можно было людям сообщить, что у них в коде нужно эти места поправить. |
|
Данные правки теперь в этом PR #36 |
2 participants
📌 Описание изменений
✅ Тип изменений
🧾 Дополнительно